Skip to content

refactor: remove unused num_partial_branches field - #73

Merged
ideepu merged 1 commit into
mainfrom
refactor/remove-num-partial-branches
Aug 1, 2026
Merged

refactor: remove unused num_partial_branches field#73
ideepu merged 1 commit into
mainfrom
refactor/remove-num-partial-branches

Conversation

@ideepu

@ideepu ideepu commented Aug 1, 2026

Copy link
Copy Markdown
Owner

Summary

  • Remove unused num_partial_branches from PytestCoverageInfo and parsing; it was never used for diff coverage or comment templates
  • Drop related fixture/test expectations and a stale BRANCH_COVERAGE TODO comment

Test plan

  • make lint
  • make test

@ideepu

ideepu commented Aug 1, 2026

Copy link
Copy Markdown
Owner Author

Coverage report

  

This PR does not include changes to coverable code or code with missing coverage.

Click to see whole project coverage
File Statements Missing Branches Missing
Coverage         

Missing stmts              

Missing branches              
  codecov
  badge.py
  config.py 98
  diff_grouper.py
  exceptions.py
  github.py 219 -> 207
  github_client.py 128, 130 127 -> 128, 129 -> 130
  groups.py
  log.py
  main.py 119-120 118 -> 119
  template.py
  codecov/coverage
  base.py 79-81, 106-108 58 -> 60
  jest.py 65-69, 73-74, 77-88, 94-106, 112-126, 145-159, 175-179, 194-212, 217 67 -> 68, 67 -> 69, 79 -> 80, 79 -> 88, 80 -> 81, 80 -> 85, 96 -> 97, 96 -> 106, 97 -> 98, 97 -> 102, 150 -> 151, 150 -> 158, 199 -> 200, 199 -> 212
  pytest.py
Project Total    

@ideepu
ideepu merged commit b874406 into main Aug 1, 2026
2 checks passed
@ideepu
ideepu deleted the refactor/remove-num-partial-branches branch August 1, 2026 23:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant